Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[#1267] fix: Add missing headers and license #1353

Merged
merged 1 commit into from
Jan 7, 2024

Conversation

brandboat
Copy link
Member

What changes were proposed in this pull request?

add missing headers and license

Why are the changes needed?

Fix: #1267

Does this PR introduce any user-facing change?

N/A

How was this patch tested?

N/A (no code change, only add missing license)

@brandboat brandboat changed the title Add missing headers and license [#1267] fix: Add missing headers and license Jan 5, 2024
@@ -1,3 +1,7 @@
/*
* Copyright 2023 Datastrato Pvt Ltd.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It's 2024 now, happy new year!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No, this file is written in 2023.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

oh

@qqqttt123 qqqttt123 added need backport Issues that need to backport to another branch branch-0.3 labels Jan 6, 2024
@qqqttt123
Copy link
Contributor

Thanks @brandboat @brandboat , merged to master & branch 0.3.

@qqqttt123 qqqttt123 merged commit 986efd9 into apache:main Jan 7, 2024
11 checks passed
github-actions bot pushed a commit that referenced this pull request Jan 7, 2024
### What changes were proposed in this pull request?
add missing headers and license

### Why are the changes needed?
Fix: #1267

### Does this PR introduce _any_ user-facing change?
N/A

### How was this patch tested?
N/A (no code change, only add missing license)
diqiu50 pushed a commit to diqiu50/gravitino that referenced this pull request Jan 8, 2024
### What changes were proposed in this pull request?
add missing headers and license

### Why are the changes needed?
Fix: apache#1267

### Does this PR introduce _any_ user-facing change?
N/A

### How was this patch tested?
N/A (no code change, only add missing license)
@brandboat brandboat deleted the add-missing-header branch January 8, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
branch-0.3 need backport Issues that need to backport to another branch
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Improvement] Files missing headers and license
3 participants